Skip to content

[mock_uss] Better slow update check #1334

Merged
BenjaminPelletier merged 4 commits intointeruss:mainfrom
Orbitalize:1330_faster_interval
Feb 18, 2026
Merged

[mock_uss] Better slow update check #1334
BenjaminPelletier merged 4 commits intointeruss:mainfrom
Orbitalize:1330_faster_interval

Conversation

@the-glu
Copy link
Contributor

@the-glu the-glu commented Feb 11, 2026

Follow #1333

Improve slow update check in mock_uss by doing a proper slow update check.

This also make notification coming faster instead of having arbitrarily in the middle of the flight.

Idea is the following:

  • Compute average update rate with a moving window of 3 seconds.
  • Check for % of update rate bellow the required rate with a moving window of 10 (over the averages update rates)

Notifications are created for each "non-compliant bucket", and placed in time at the end of the bucket, making them more realistic. However, windows spans are quite arbitrary.

@the-glu the-glu force-pushed the 1330_faster_interval branch from 4f5c32c to df517da Compare February 11, 2026 12:49
@the-glu the-glu force-pushed the 1330_faster_interval branch from df517da to 0b59e5b Compare February 11, 2026 17:46
BenjaminPelletier
BenjaminPelletier approved these changes Feb 12, 2026
View reviewed changes
the-glu and others added 2 commits February 17, 2026 17:19
@the-glu @BenjaminPelletier
Apply suggestion from @BenjaminPelletier
9d56299 
Co-authored-by: Benjamin Pelletier <BenjaminPelletier@users.noreply.github.com>
@the-glu
Switch to constant for moving windows
cfcc7d6
@BenjaminPelletier
Copy link
Member

BenjaminPelletier commented Feb 18, 2026

Note: the errors in an earlier failed CI run for cfcc7d6 looked like the below. Retrying the failed jobs made them succeed without changing anything.

Fetching submodules
  /usr/bin/git submodule sync
  /usr/bin/git -c protocol.version=2 submodule update --init --force --depth=1
  Submodule 'interfaces/astm-utm/Protocol' (https://github.com/astm-utm/Protocol) registered for path 'interfaces/astm-utm/Protocol'
  Submodule 'interfaces/automated_testing' (https://github.com/interuss/automated_testing_interfaces) registered for path 'interfaces/automated_testing'
  Submodule 'interfaces/rid/v1' (https://github.com/uastech/standards) registered for path 'interfaces/rid/v1'
  Submodule 'interfaces/rid/v2' (https://github.com/uastech/standards) registered for path 'interfaces/rid/v2'
  Cloning into '/home/runner/work/monitoring/monitoring/interfaces/astm-utm/Protocol'...
  Cloning into '/home/runner/work/monitoring/monitoring/interfaces/automated_testing'...
  Cloning into '/home/runner/work/monitoring/monitoring/interfaces/rid/v1'...
  Cloning into '/home/runner/work/monitoring/monitoring/interfaces/rid/v2'...
  From https://github.com/astm-utm/Protocol
   * branch            63ba6c5ff9c9abb93c55fbdf2bffc0b089d10c4a -> FETCH_HEAD
  Submodule path 'interfaces/astm-utm/Protocol': checked out '63ba6c5ff9c9abb93c55fbdf2bffc0b089d10c4a'
  Error: fatal: could not read Username for 'https://github.com/': terminal prompts disabled
  Error: fatal: the remote end hung up unexpectedly
  Error: fatal: Fetched in submodule path 'interfaces/automated_testing', but it did not contain 3e6060bd6d5cd665eb5cc60ec7aaeab104547769. Direct fetching of that commit failed.
  Error: The process '/usr/bin/git' failed with exit code 128

@BenjaminPelletier BenjaminPelletier merged commit 6c94924 into interuss:main Feb 18, 2026
36 of 39 checks passed
github-actions bot added a commit that referenced this pull request Feb 18, 2026
@github-actions
[mock_uss] Better slow update check (#1334)
724097b 
Co-authored-by: Benjamin Pelletier <BenjaminPelletier@users.noreply.github.com> 6c94924
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@BenjaminPelletier BenjaminPelletier BenjaminPelletier approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@the-glu @BenjaminPelletier

Comments